[INFO] cloning repository https://github.com/qisanyijiu/kvs
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/qisanyijiu/kvs" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fqisanyijiu%2Fkvs", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fqisanyijiu%2Fkvs'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 711e05357d42b1bc3ccae767a9a378df8f1b6e12
[INFO] checking qisanyijiu/kvs against try#83ec5b3aeb5925d43b5be7b50c090f2f0336fcc1 for pr-134021
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fqisanyijiu%2Fkvs" "/workspace/builds/worker-6-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-6-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/qisanyijiu/kvs
[INFO] finished tweaking git repo https://github.com/qisanyijiu/kvs
[INFO] tweaked toml for git repo https://github.com/qisanyijiu/kvs written to /workspace/builds/worker-6-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/qisanyijiu/kvs on toolchain 83ec5b3aeb5925d43b5be7b50c090f2f0336fcc1
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+83ec5b3aeb5925d43b5be7b50c090f2f0336fcc1" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/qisanyijiu/kvs already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+83ec5b3aeb5925d43b5be7b50c090f2f0336fcc1" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:3a6becf2bc8dde7f3fa57ede90e4f284e72d296796fc446bbb1e2c7cc0530151" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] b2b706134e1322bbc7522d655aa7cf45cdad18ac674d05bceb6f248283a25792
[INFO] running `Command { std: "docker" "start" "b2b706134e1322bbc7522d655aa7cf45cdad18ac674d05bceb6f248283a25792", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "b2b706134e1322bbc7522d655aa7cf45cdad18ac674d05bceb6f248283a25792" "/opt/rustwide/cargo-home/bin/cargo" "+83ec5b3aeb5925d43b5be7b50c090f2f0336fcc1" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "b2b706134e1322bbc7522d655aa7cf45cdad18ac674d05bceb6f248283a25792", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-w" "/opt/rustwide/workdir" "--user" "0:0" "b2b706134e1322bbc7522d655aa7cf45cdad18ac674d05bceb6f248283a25792" "/opt/rustwide/cargo-home/bin/cargo" "+83ec5b3aeb5925d43b5be7b50c090f2f0336fcc1" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling libc v0.2.119
[INFO] [stderr]    Compiling syn v1.0.86
[INFO] [stderr]    Compiling log v0.4.14
[INFO] [stderr]    Compiling memoffset v0.5.6
[INFO] [stderr]    Compiling crossbeam-utils v0.8.7
[INFO] [stderr]    Compiling memoffset v0.6.5
[INFO] [stderr]    Compiling crossbeam-epoch v0.9.7
[INFO] [stderr]    Compiling crossbeam-epoch v0.8.2
[INFO] [stderr]    Compiling rayon-core v1.9.1
[INFO] [stderr]    Compiling quote v1.0.15
[INFO] [stderr]    Compiling arrayvec v0.4.12
[INFO] [stderr]     Checking either v1.6.1
[INFO] [stderr]     Checking nodrop v0.1.14
[INFO] [stderr]    Compiling rayon v1.5.1
[INFO] [stderr]    Compiling semver v1.0.6
[INFO] [stderr]     Checking crossbeam-queue v0.2.3
[INFO] [stderr]     Checking unicode-width v0.1.9
[INFO] [stderr]    Compiling syn v0.15.44
[INFO] [stderr]     Checking regex v1.5.4
[INFO] [stderr]     Checking textwrap v0.11.0
[INFO] [stderr]     Checking crossbeam-utils v0.6.6
[INFO] [stderr]    Compiling num-traits v0.2.14
[INFO] [stderr]     Checking vec_map v0.8.2
[INFO] [stderr]     Checking strsim v0.8.0
[INFO] [stderr]    Compiling unicode-segmentation v1.9.0
[INFO] [stderr]     Checking crossbeam-utils v0.6.5 (https://github.com/crossbeam-rs/crossbeam.git?rev=8cc906b#8cc906b9)
[INFO] [stderr]     Checking fxhash v0.2.1
[INFO] [stderr]     Checking memoffset v0.2.1
[INFO] [stderr]     Checking historian v4.0.4
[INFO] [stderr]     Checking crossbeam-channel v0.4.4
[INFO] [stderr]     Checking crossbeam-deque v0.7.4
[INFO] [stderr]     Checking scopeguard v0.3.3
[INFO] [stderr]     Checking predicates-core v1.0.3
[INFO] [stderr]    Compiling escargot v0.4.0
[INFO] [stderr]     Checking regex-automata v0.1.10
[INFO] [stderr]     Checking itertools v0.8.2
[INFO] [stderr]     Checking csv-core v0.1.10
[INFO] [stderr]     Checking normalize-line-endings v0.3.0
[INFO] [stderr]     Checking termtree v0.2.4
[INFO] [stderr]     Checking crossbeam v0.7.3
[INFO] [stderr]     Checking same-file v1.0.6
[INFO] [stderr]    Compiling rustc_version v0.4.0
[INFO] [stderr]     Checking difference v2.0.0
[INFO] [stderr]     Checking crossbeam-epoch v0.7.2
[INFO] [stderr]     Checking crossbeam-channel v0.5.2
[INFO] [stderr]     Checking crossbeam-epoch v0.7.1 (https://github.com/crossbeam-rs/crossbeam.git?rev=8cc906b#8cc906b9)
[INFO] [stderr]     Checking predicates-tree v1.0.5
[INFO] [stderr]     Checking walkdir v2.3.2
[INFO] [stderr]     Checking rand_xoshiro v0.1.0
[INFO] [stderr]     Checking remove_dir_all v0.5.3
[INFO] [stderr]     Checking fastrand v1.7.0
[INFO] [stderr]    Compiling cast v0.2.7
[INFO] [stderr]    Compiling heck v0.3.3
[INFO] [stderr]     Checking panic-control v0.1.4
[INFO] [stderr]     Checking iovec v0.1.4
[INFO] [stderr]     Checking num_cpus v1.13.1
[INFO] [stderr]     Checking net2 v0.2.37
[INFO] [stderr]     Checking parking_lot_core v0.6.2
[INFO] [stderr]     Checking atty v0.2.14
[INFO] [stderr]     Checking fs2 v0.4.3
[INFO] [stderr]     Checking crossbeam-deque v0.8.1
[INFO] [stderr]     Checking clap v2.34.0
[INFO] [stderr]     Checking bytes v0.4.12
[INFO] [stderr]     Checking backtrace v0.3.64
[INFO] [stderr]     Checking rand_os v0.1.3
[INFO] [stderr]     Checking crossbeam-skiplist v0.0.0 (https://github.com/crossbeam-rs/crossbeam.git?rev=8cc906b#8cc906b9)
[INFO] [stderr]     Checking tokio-threadpool v0.1.18
[INFO] [stderr]     Checking tempfile v3.3.0
[INFO] [stderr]     Checking parking_lot v0.9.0
[INFO] [stderr]     Checking rand v0.6.5
[INFO] [stderr]     Checking tokio-io v0.1.13
[INFO] [stderr]     Checking tokio-serde v0.3.0
[INFO] [stderr]     Checking mio v0.6.23
[INFO] [stderr]     Checking float-cmp v0.8.0
[INFO] [stderr]     Checking env_logger v0.6.2
[INFO] [stderr]     Checking predicates v1.0.8
[INFO] [stderr]     Checking tokio-codec v0.1.2
[INFO] [stderr]     Checking tokio-fs v0.1.7
[INFO] [stderr]     Checking criterion-plot v0.3.1
[INFO] [stderr]     Checking tokio-reactor v0.1.12
[INFO] [stderr]     Checking mio-uds v0.6.8
[INFO] [stderr]     Checking tokio-udp v0.1.6
[INFO] [stderr]     Checking tokio-tcp v0.1.4
[INFO] [stderr]     Checking tokio-uds v0.2.7
[INFO] [stderr]    Compiling structopt-derive v0.2.18
[INFO] [stderr]     Checking tokio v0.1.22
[INFO] [stderr]     Checking structopt v0.2.18
[INFO] [stderr]    Compiling synstructure v0.12.6
[INFO] [stderr]    Compiling serde_derive v1.0.136
[INFO] [stderr]    Compiling failure_derive v0.1.8
[INFO] [stderr]     Checking failure v0.1.8
[INFO] [stderr]     Checking serde v1.0.136
[INFO] [stderr]     Checking serde_json v1.0.79
[INFO] [stderr]     Checking bincode v1.3.3
[INFO] [stderr]     Checking hashbrown v0.2.2
[INFO] [stderr]     Checking serde_bytes v0.10.5
[INFO] [stderr]     Checking bstr v0.2.17
[INFO] [stderr]     Checking csv v1.1.6
[INFO] [stderr]     Checking pagecache v0.15.0
[INFO] [stderr]     Checking tokio-serde-json v0.2.0
[INFO] [stderr]     Checking tinytemplate v1.2.1
[INFO] [stderr]     Checking criterion v0.2.11
[INFO] [stderr]     Checking assert_cmd v0.11.1
[INFO] [stderr]     Checking sled v0.22.1
[INFO] [stderr]     Checking kvs v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: field `path` is never read
[INFO] [stdout]   --> src/engines/kvs.rs:46:5
[INFO] [stdout]    |
[INFO] [stdout] 44 | pub struct KvStore<P: ThreadPool> {
[INFO] [stdout]    |            ------- field in this struct
[INFO] [stdout] 45 |     // directory for the log and other data
[INFO] [stdout] 46 |     path: Arc<PathBuf>,
[INFO] [stdout]    |     ^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `KvStore` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/engines/kvs.rs:249:13
[INFO] [stdout]     |
[INFO] [stdout] 249 |             serde_json::to_writer(&mut self.writer, &cmd);
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]     = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 249 |             let _ = serde_json::to_writer(&mut self.writer, &cmd);
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]   --> src/engines/sled.rs:66:17
[INFO] [stdout]    |
[INFO] [stdout] 66 |                 db.flush();
[INFO] [stdout]    |                 ^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]    |
[INFO] [stdout] 66 |                 let _ = db.flush();
[INFO] [stdout]    |                 +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/error.rs:5:17
[INFO] [stdout]   |
[INFO] [stdout] 5 | #[derive(Debug, Fail)]
[INFO] [stdout]   |                 ^---
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `Fail` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_KvsError`
[INFO] [stdout] 6 | pub enum KvsError {
[INFO] [stdout]   |          -------- `KvsError` is not local
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: `#[warn(non_local_definitions)]` on by default
[INFO] [stdout]   = note: this warning originates in the derive macro `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/error.rs:5:17
[INFO] [stdout]   |
[INFO] [stdout] 5 | #[derive(Debug, Fail)]
[INFO] [stdout]   |                 ^---
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `Display` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_KvsError`
[INFO] [stdout] 6 | pub enum KvsError {
[INFO] [stdout]   |          -------- `KvsError` is not local
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: this warning originates in the derive macro `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `path` is never read
[INFO] [stdout]   --> src/engines/kvs.rs:46:5
[INFO] [stdout]    |
[INFO] [stdout] 44 | pub struct KvStore<P: ThreadPool> {
[INFO] [stdout]    |            ------- field in this struct
[INFO] [stdout] 45 |     // directory for the log and other data
[INFO] [stdout] 46 |     path: Arc<PathBuf>,
[INFO] [stdout]    |     ^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `KvStore` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]    --> src/engines/kvs.rs:249:13
[INFO] [stdout]     |
[INFO] [stdout] 249 |             serde_json::to_writer(&mut self.writer, &cmd);
[INFO] [stdout]     |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]     = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 249 |             let _ = serde_json::to_writer(&mut self.writer, &cmd);
[INFO] [stdout]     |             +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `std::result::Result` that must be used
[INFO] [stdout]   --> src/engines/sled.rs:66:17
[INFO] [stdout]    |
[INFO] [stdout] 66 |                 db.flush();
[INFO] [stdout]    |                 ^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]    |
[INFO] [stdout] 66 |                 let _ = db.flush();
[INFO] [stdout]    |                 +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/error.rs:5:17
[INFO] [stdout]   |
[INFO] [stdout] 5 | #[derive(Debug, Fail)]
[INFO] [stdout]   |                 ^---
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `Fail` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_KvsError`
[INFO] [stdout] 6 | pub enum KvsError {
[INFO] [stdout]   |          -------- `KvsError` is not local
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: `#[warn(non_local_definitions)]` on by default
[INFO] [stdout]   = note: this warning originates in the derive macro `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/error.rs:5:17
[INFO] [stdout]   |
[INFO] [stdout] 5 | #[derive(Debug, Fail)]
[INFO] [stdout]   |                 ^---
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `Display` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_KvsError`
[INFO] [stdout] 6 | pub enum KvsError {
[INFO] [stdout]   |          -------- `KvsError` is not local
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: this warning originates in the derive macro `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 25.63s
[INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: criterion v0.2.11, pagecache v0.15.0
[INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 2`
[INFO] running `Command { std: "docker" "inspect" "b2b706134e1322bbc7522d655aa7cf45cdad18ac674d05bceb6f248283a25792", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "b2b706134e1322bbc7522d655aa7cf45cdad18ac674d05bceb6f248283a25792", kill_on_drop: false }`
[INFO] [stdout] b2b706134e1322bbc7522d655aa7cf45cdad18ac674d05bceb6f248283a25792
